Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add DevContainers support for editing project locally #1506

Merged
merged 3 commits into from
Mar 25, 2025

Conversation

NyanKiyoshi
Copy link
Member

@NyanKiyoshi NyanKiyoshi commented Mar 20, 2025

This adds support for remote development using DevContainers, as well as plain-Dockerfile and docker-compose.yaml stack for containerized local development (with or without devcontainers, e.g., using docker-compose up command manually).

Warning

It may be unstable, NPM often complains about package-lock.json file being outdated in main branch, something or someone may be pushing broken package-lock.json files which may prevent the docker-compose stack from running (using npm ci)

This adds a Dockerfile and docker-compose.yaml in order to allow containerized local development.
@NyanKiyoshi NyanKiyoshi added the enhancement New feature or request label Mar 20, 2025
@NyanKiyoshi NyanKiyoshi requested a review from a team March 20, 2025 14:44
@NyanKiyoshi NyanKiyoshi self-assigned this Mar 20, 2025
Copy link

vercel bot commented Mar 20, 2025

@NyanKiyoshi is attempting to deploy a commit to the Saleor Commerce Team on Vercel.

A member of the Team first needs to authorize it.

@@ -1,4 +1,5 @@
{
"name": "saleor-docs",
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needed otherwise the package-lock.json file will change the application name from saleor-docs to app as NPM was trying to guess the name of the project

@NyanKiyoshi
Copy link
Member Author

After some internal feedback, we will changing the pull request to add support for devcontainers instead

@NyanKiyoshi NyanKiyoshi marked this pull request as draft March 20, 2025 16:17
@NyanKiyoshi NyanKiyoshi removed the request for review from a team March 20, 2025 16:17
@NyanKiyoshi NyanKiyoshi changed the title Add docker-compose for running locally Add DevContainers support for editing project locally Mar 25, 2025
@NyanKiyoshi NyanKiyoshi marked this pull request as ready for review March 25, 2025 12:28
@NyanKiyoshi NyanKiyoshi requested a review from a team March 25, 2025 12:28
Copy link

vercel bot commented Mar 25, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
saleor-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 25, 2025 1:56pm

@aniav aniav merged commit 11944a7 into saleor:main Mar 25, 2025
15 of 16 checks passed
@NyanKiyoshi NyanKiyoshi deleted the add-containers branch March 25, 2025 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Licenses Reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants